Skip to content

Fix openeo process - #81

Merged
firzaariany merged 2 commits into
mainfrom
fix-openeo-process
Sep 11, 2026
Merged

firzaariany merged 2 commits into
mainfrom
fix-openeo-process

Conversation

@firzaariany

Copy link
Copy Markdown
Contributor

Closes #69 and #73

  • Exported UDP from vegetation condition index and oil spill detection notebooks
  • Removed Python syntaxes to dynamically pick temporal window in vegetation condition index notebooks
  • Does not require review since the changes are minimal and the science behind custom script -> openEO process is preserved

firzaariany and others added 2 commits September 11, 2026 14:48
Splits the single time range into time_before, time_oil_spill,
time_dissolute, and time_after parameters, loading each acquisition
window independently so the exported UDP stays reusable and derives
its process graph from the oil_spill_image cube (time_oil_spill)
instead of a hardcoded reduction.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Replace anchor_date/year_from/year_to/tolerance_days with a live `time`
parameter and a build-time `nb_past_years` knob, mirroring
ndvi_anomaly_detection's pattern: historical windows are derived via
array_element/date_shift graph nodes instead of Python date math, and
the exported UDP only declares bounding_box/time as runtime parameters
since cloud_masking_method and nb_past_years never appear as live
from_parameter references in the graph.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@firzaariany
firzaariany merged commit f268bdf into main Sep 11, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

VCI notebook bakes date-window logic into Python. UDP time parameter can't be reused across calls

1 participant